Add wstd::main and wstd::test macros for WASIp3 - #149
Open
adamrk wants to merge 6 commits into
Open
Conversation
adamrk
commented
Sep 3, 2026
| use wstd::http::{Body, Client, Request, error::ErrorCode}; | ||
|
|
||
| #[wstd::main] | ||
| #[wstd::test] |
Contributor
Author
There was a problem hiding this comment.
A drive-by fix. This test wasn't actually running under cargo test because it used the main macro.
adamrk
force-pushed
the
abk/main-macro-for-wasip3
branch
from
September 3, 2026 14:25
af9bae0 to
cabda1f
Compare
pchickey
reviewed
Sep 8, 2026
| # * AWS auth environment variables, for running the wstd-aws integration tests. | ||
| # * . directory is available at . | ||
| runner = "wasmtime run -Shttp --env AWS_ACCESS_KEY_ID --env AWS_SECRET_ACCESS_KEY --env AWS_SESSION_TOKEN --dir .::." | ||
| runner = "wasmtime run -Sp3 -Shttp --env AWS_ACCESS_KEY_ID --env AWS_SECRET_ACCESS_KEY --env AWS_SESSION_TOKEN --dir .::." |
Contributor
There was a problem hiding this comment.
Transition to using wasm32-wasip3 target should mean we have a separate section with this changed runner, and can exclude p3 stuff on the p2 target
pchickey
reviewed
Sep 8, 2026
| ::core::unreachable!( | ||
| "wstd p3 components run via the wasi:cli/run@0.3.0 export, not `main`" | ||
| ) | ||
| } |
Contributor
There was a problem hiding this comment.
I assume this breaks in wasm32-wasip3 target
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Based on #146
Addresses #148